From: Keir Fraser <keir.fraser@citrix.com>
Date: Mon, 13 Oct 2008 12:15:20 +0000 (+0100)
Subject: x86: propagate return value of alloc_l1_table()
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~14083
X-Git-Url: https://dgit.raspbian.org/%22http:/www.example.com/cgi/%22https:/%22bookmarks://%22Dat/%22http:/www.example.com/cgi/%22https:/%22bookmarks:/%22Dat?a=commitdiff_plain;h=b525c05cf95fa9528cfb83ecc120087881013980;p=xen.git

x86: propagate return value of alloc_l1_table()

A blatant mistake of mine resulted in the return value of
alloc_l1_table() to be ignored with the preemptable page table update
changes.

Signed-off-by: Jan Beulich <jbeulich@novell.com>
---

diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c
index f3082eeedd..4c8b40a901 100644
--- a/xen/arch/x86/mm.c
+++ b/xen/arch/x86/mm.c
@@ -1883,8 +1883,7 @@ static int alloc_page_type(struct page_info *page, unsigned long type,
     switch ( type & PGT_type_mask )
     {
     case PGT_l1_page_table:
-        alloc_l1_table(page);
-        rc = 0;
+        rc = alloc_l1_table(page);
         break;
     case PGT_l2_page_table:
         rc = alloc_l2_table(page, type, preemptible);